Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixing modify alertmanager yaml #44

Merged
merged 18 commits into from
Jan 19, 2024
Merged

Conversation

TehreemNisa
Copy link
Contributor

No description provided.

Copy link

@TehreemNisa Yikes! You better fix it before anyone else finds out! Build has Failed!

Copy link

@TehreemNisa Yikes! You better fix it before anyone else finds out! Build has Failed!

Copy link

@TehreemNisa Yikes! You better fix it before anyone else finds out! Build has Failed!

Copy link

@TehreemNisa Yikes! You better fix it before anyone else finds out! Build has Failed!

Copy link

@TehreemNisa Yikes! You better fix it before anyone else finds out! Build has Failed!

Copy link

@TehreemNisa Image is available for testing. docker pull ghcr.io/stakater/grafana-cloud-ansible-operator:snapshot-pr-44-b1c315fb

Copy link

@TehreemNisa Image is available for testing. docker pull ghcr.io/stakater/grafana-cloud-ansible-operator:snapshot-pr-44-a16debb9

Copy link

@TehreemNisa Image is available for testing. docker pull ghcr.io/stakater/grafana-cloud-ansible-operator:snapshot-pr-44-04e8315a

This reverts commit 04e8315.
Copy link

@TehreemNisa Image is available for testing. docker pull ghcr.io/stakater/grafana-cloud-ansible-operator:snapshot-pr-44-e0ec80e2

Copy link

@TehreemNisa Image is available for testing. docker pull ghcr.io/stakater/grafana-cloud-ansible-operator:snapshot-pr-44-d5227327

Copy link

@TehreemNisa Yikes! You better fix it before anyone else finds out! Build has Failed!

Copy link

@TehreemNisa Yikes! You better fix it before anyone else finds out! Build has Failed!

Copy link

@TehreemNisa Image is available for testing. docker pull ghcr.io/stakater/grafana-cloud-ansible-operator:snapshot-pr-44-4d748118

Copy link

@TehreemNisa Image is available for testing. docker pull ghcr.io/stakater/grafana-cloud-ansible-operator:snapshot-pr-44-c556bae5

Copy link

@TehreemNisa Image is available for testing. docker pull ghcr.io/stakater/grafana-cloud-ansible-operator:snapshot-pr-44-46f043a8

Copy link

@TehreemNisa Image is available for testing. docker pull ghcr.io/stakater/grafana-cloud-ansible-operator:snapshot-pr-44-cbbc13bd

Copy link

@TehreemNisa Image is available for testing. docker pull ghcr.io/stakater/grafana-cloud-ansible-operator:snapshot-pr-44-9cee8730

bnallapeta
bnallapeta previously approved these changes Jan 18, 2024
Copy link

@bnallapeta bnallapeta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good except for a minor change.

Comment on lines 26 to 28
- name: Print Provision Mode
ansible.builtin.debug:
msg: "Ansible Version: {{ provision_mode }}"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this was added only for debugging purpose right? Remove if not required.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Updated

Copy link

@TehreemNisa Image is available for testing. docker pull ghcr.io/stakater/grafana-cloud-ansible-operator:snapshot-pr-44-53056102

@bnallapeta bnallapeta merged commit 34e4ae9 into main Jan 19, 2024
6 checks passed
@bnallapeta bnallapeta deleted the fix-alertmanager-modify-task branch January 19, 2024 05:20
Copy link

@bnallapeta Image is available for testing. docker pull ghcr.io/stakater/grafana-cloud-ansible-operator:v0.0.30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants